The CI error should be unrelated:

  docker-compose up -d ${DIALECT}
  shell: /usr/bin/bash -e {0}
  env:
    DIALECT: postgres
    SEQ_PORT: 54320
/home/runner/work/_temp/4024a7e3-607c-4ffe-b82d-9ae14520d99c.sh: line 1: docker-compose: command not found
Error: Process completed with exit code 127.

CI seems to be broken atm. It uses ubuntu-latest and it wasn't written for Ubuntu 24.04 and the new Docker version. It should use docker compose (space instead of -). That should be fixed as well before we can merge this.

Even with that; in the new version of the CLI we're using ansis. Can we use that here as well?
https://github.com/sequelize/sequelize/blob/2a6590352e6f3d31f15bddf54a9a8a9f83312600/packages/cli/package.json#L14

CI seems to be broken atm. It uses ubuntu-latest and it wasn't written for Ubuntu 24.04 and the new Docker version. It should use docker compose (space instead of -). That should be fixed as well before we can merge this

I'm not too familiar with the codebase yet, where should I change?

Even with that; in the new version of the CLI we're using ansis.

Didn't notice that, I'll try to replace ansis at the same time. By the way, why there're two libraries for cli coloring in the first place? 😅

The CI for PRs is here; https://github.com/sequelize/cli/blob/main/.github%2Fworkflows%2Fci.yml

With the new version of the CLI, I meant the rewrite that we are doing over in the sequelize/sequelize repo for v7. This sequelize-cli library will be deprecated once that comes out.

The CI for PRs is here; main/.github%2Fworkflows%2Fci.yml

With the new version of the CLI, I meant the rewrite that we are doing over in the sequelize/sequelize repo for v7. This sequelize-cli library will be deprecated once that comes out.

I already tried at 433846a (#1532)

And there is a new error:

Run docker run --link ${DIALECT}:db -e CHECK_PORT=${SEQ_PORT::-1} -e CHECK_HOST=db --net cli_default giorgos/takis
  docker run --link ${DIALECT}:db -e CHECK_PORT=${SEQ_PORT::-1} -e CHECK_HOST=db --net cli_default giorgos/takis
  shell: /usr/bin/bash -e {0}
  env:
    DIALECT: mysql
    SEQ_PORT: 33060
Unable to find image 'giorgos/takis:latest' locally
latest: Pulling from giorgos/takis
docker: [DEPRECATION NOTICE] Docker Image Format v1 and Docker Image manifest version 2, schema 1 support is disabled by default and will be removed in an upcoming release. Suggest the author of docker.io/giorgos/takis:latest to upgrade the image to the OCI Format or Docker Image manifest v2, schema 2. More information at https://docs.docker.com/go/deprecated-image-specs/.
See 'docker run --help'.
Error: Process completed with exit code 125.

Hello @WikiRik,

this PR is a duplicate.
I have already suggested the PR refactor: switch cli-color to ansis.
But the PR was closed with the reason:

I'll close this PR since we have no need to switch here.

Hello @WikiRik,

this PR is a duplicate. I have already suggested the PR refactor: switch cli-color to ansis. But the PR was closed with the reason:

I'll close this PR since we have no need to switch here.

@webdiscus I believe that's because Node 10 is not supported in your library?

Hello @WikiRik,
this PR is a duplicate. I have already suggested the PR refactor: switch cli-color to ansis. But the PR was closed with the reason:

I'll close this PR since we have no need to switch here.

@webdiscus I believe that's because Node 10 is not supported in your library?

You are right:

For v6 (the current version) we will keep supporting Node 10, so I'll close this PR since we have no need to switch here.

In this case, Picocolors is the best choice.

Hello everyone,

Dropping cli-color in favor of something else that doesn't have es5-ext as a dependency will benefit a lot of users still using v6 so I fixed the CI process - please check #1533, there were 2 extra small fixes besides changing docker-compose to docker compose

Thanks everybody for the input. If/when the CI passes, I will approve this PR and see if I can get a new release out.

🎉 This PR is included in version 6.6.3 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀